Skip to content

fix: update Content Health Monitor setup instructions for new Connect UI#345

Open
ChaitaC wants to merge 2 commits intomainfrom
chaita/fix-setup-instructions-vars-tab-344
Open

fix: update Content Health Monitor setup instructions for new Connect UI#345
ChaitaC wants to merge 2 commits intomainfrom
chaita/fix-setup-instructions-vars-tab-344

Conversation

@ChaitaC
Copy link
Copy Markdown

@ChaitaC ChaitaC commented Apr 20, 2026

Summary

The Connect UI was redesigned — the standalone "Vars" tab was replaced with an "Advanced" tab containing an Environment Variables section, and toolbar buttons are now labeled. The Content Health Monitor's setup instructions and screenshots were outdated.

Changes

  • "Vars" tab"Advanced" tab
  • "gear icon""Settings" (now a labeled button)
  • "Add Variable""+ Add"
  • "Refresh Report""Refresh"
  • "View Logs""Logs"
  • Replaced old screenshots (vars.png, settings-gear-icon.png) with new ones (env-vars.png, settings-button.png, updated refresh-report.png)
  • Updated manifest.json with new file entries and checksums

Closes #344

Note

There are similar (minor) stale UI references in other extensions — will file a separate cleanup issue for those.

The Connect settings drawer replaced the standalone "Vars" tab with an
"Advanced" tab containing an Environment Variables section. Update the
setup instructions and screenshots to match the current UI.

Closes #344

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 20, 2026

Extension release summary

🚀 Will release on merge

  • content-health-monitor: 1.0.01.0.1

See the contributing guide for details.

Copy link
Copy Markdown
Collaborator

@jonkeane jonkeane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could just change all of these to new settings (might be another nudge to get folks to upgrade "look at all this new stuff"), but if this is rendering on the server, we should be able to detect which version of connect someone is on and show the old settings if they are on an older version and the new ones if they are on a newer one, yeah?

@ChaitaC
Copy link
Copy Markdown
Author

ChaitaC commented Apr 21, 2026

We could just change all of these to new settings (might be another nudge to get folks to upgrade "look at all this new stuff"), but if this is rendering on the server, we should be able to detect which version of connect someone is on and show the old settings if they are on an older version and the new ones if they are on a newer one, yeah?

Good idea! Version detection is possible via the /v1/server_settings API (it returns a version field). The extension already has a Connect client, so we could query it during setup.

The tricky part is that even within the same Connect version, individual users can toggle between old and new settings via the preview feature (userPreviewNewSettings), so server version alone doesn't tell us which UI they're seeing.

Since minimumConnectVersion is already 2025.04.0 and the new drawer defaults to on, I think targeting the new UI is the right call here. The old settings pane will eventually be removed, and this gives users on the old UI a nudge to switch over.

Copy link
Copy Markdown
Contributor

@karawoo karawoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had the same thought as @jonkeane initially but I'im ok with the simpler approach of just showing the new UI.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is the place to update in order to make sure your changes get released

Use client.version to show new UI instructions (Settings button,
Advanced tab) on Connect >= 2026.03.0 and old UI instructions
(gear icon, Vars tab) on older versions. Bumps version to 1.0.1.

Closes #344

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ChaitaC
Copy link
Copy Markdown
Author

ChaitaC commented Apr 24, 2026

Thanks @karawoo for the review and the version bump catch! I know you and @jonkeane were both leaning towards version detection, so I tried to implement it before settling on just targeting the new UI.

Here's a summary of the options I explored — I've pushed Option 2 as a prototype, but my lean is Option 1. Happy to keep Option 2 if you all think the version detection is worth it.

Option 1: New UI only (current/initial PR approach)
Replace all old UI references with the new UI.

✅ Simple, no additional logic
⚠️ Users on Connect < 2026.03.0 (down to our minimum supported 2025.04.0) will see instructions that don’t match their UI

Option 2: Server version detection (prototype implementation for evaluation)
Check client.version — if ≥ 2026.03.0 show new UI instructions, otherwise show old.

✅ No extra API calls (SDK already fetches /server_settings)
✅ Works for the vast majority of users
⚠️ Edge cases:
2026.02.x users who manually opted into new UI
2026.03.0+ users who opted out
⚠️ Requires maintaining both old + new screenshots/instructions (temporary overhead)

Option 3: User preference detection (GET /me)
Check preferences.user_preview_new_settings.

✅ 100% accurate
❌ Extra API call
❌ Outside v1 API surface
❌ Added complexity for relatively small edge cases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Content Health Monitor setup instructions reference outdated 'Vars' tab

3 participants